Repository navigation
Conversation
decision-model-lintersdk
decision-model-linter: 3 calls on 3 files, 3,951 input tokens, $0.00017, model typesafe/jev-1.13-20260917 |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 81c905a. Configure here.
|
|
||
| if ($options !== null) { | ||
| ($options->getDummyCallback())($event); | ||
| } |
There was a problem hiding this comment.
User callback invoked without try/catch
High Severity
The user-provided dummy_callback Options callback is invoked while applying the scope to an event with no surrounding try/catch. A throw from that callback escapes applyToEvent into the host app instead of being logged and swallowed, unlike sibling callbacks such as before_send.
Triggered by learned rule: Wrap user callbacks in try/catch
Reviewed by Cursor Bugbot for commit 81c905a. Configure here.
This PR tests if the decision linter will raise an error when a callback is invoked without a proper try/catch